Skip to content

Consolidate trigger SVG paths#148

Merged
DirtyF merged 2 commits intojekyll:masterfrom
ashmaroli:trigger-svg
Apr 10, 2018
Merged

Consolidate trigger SVG paths#148
DirtyF merged 2 commits intojekyll:masterfrom
ashmaroli:trigger-svg

Conversation

@ashmaroli
Copy link
Member

@ashmaroli ashmaroli commented Jul 24, 2017

  • Consolidate multiple SVG paths into one compound path (one long line)
    • minor SVG size reduction
    • reduced no. of lines in header.html
  • remove svg path fill (handled by CSS)
  • no visual change

@DirtyF DirtyF merged commit 35ac6b2 into jekyll:master Apr 10, 2018
@jekyllbot jekyllbot mentioned this pull request Apr 10, 2018
Merged
@ashmaroli ashmaroli deleted the trigger-svg branch April 10, 2018 16:05
@jekyll jekyll locked and limited conversation to collaborators Apr 30, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

@DirtyF DirtyF DirtyF approved these changes

+1 more reviewer

@pathawks pathawks pathawks approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

frozen-due-to-age

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@ashmaroli @DirtyF @pathawks @jekyllbot